Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding Validatus Services #50

Merged
merged 15 commits into from
Dec 21, 2024
Merged

Conversation

Validatus
Copy link
Contributor

This pull request includes the addition of Validatus service links, such as RPC endpoints, seed nodes, namadillo, snapshots, indexer, masp-indexer.

Add Validatus interfaces.json
Add Validatus namada-indexers.json
Add Validatus peers.json
Add Validatus seeds.json
Add Validatus snapshots.json
@Crypto-Druide
Copy link
Contributor

Hello @Validatus ,
Thanks for your support !
I noticed that you created new entries for your seed and peer addresses, even though they were already in the files.
Since I can't edit your files and your new entries are correct, could you please update your PR to remove the previous entries?
Thank you and have a nice day.

@Validatus
Copy link
Contributor Author

I noticed that you created new entries for your seed and peer addresses, even though they were already in the files.
That's perfect, you're absolutely right. Thank you for pointing that out! Let's stay strong together! 😊

@Crypto-Druide
Copy link
Contributor

Hello @Validatus ,
There is a syntax error in the namada-indexers.json file. (You forgot the closing brace).
Please see:
image

Thanks for the hint!
@Validatus
Copy link
Contributor Author

Thanks for the hint.
Next time I'll definitely use my IDE - maybe it can also proofread my life decisions while we're at it! 🤖

And let's don't forget: Brackets are like relationships - they need a proper closure!
Thanks @Crypto-Druide

@Crypto-Druide
Copy link
Contributor

Hey @Validatus
I'm sorry to ask you again, but the structure is not correct in the other files.
Could you please check and make the necessary corrections?
Thank you and have a nice day.

@Validatus
Copy link
Contributor Author

Validatus commented Dec 19, 2024

We will do so until it's fixed.
Updated JSON files.

Thanks again!

@Validatus
Copy link
Contributor Author

We are implementing a linter now as well.

@Validatus
Copy link
Contributor Author

Not my day at all.... @Crypto-Druide!
Thanks...

@Validatus
Copy link
Contributor Author

When Errors Persist - Punish Me! Close the PR and I will redo all of it.

Thanks @Crypto-Druide

@Crypto-Druide Crypto-Druide merged commit 56e0b37 into Luminara-Hub:main Dec 21, 2024
@Validatus
Copy link
Contributor Author

Thanks @Crypto-Druide

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants